Repository navigation
chore(deps): update .NET dependencies - #145
deniswsrosa wants to merge 2 commits into
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates several NuGet package dependencies across the API and Integration Tests projects, and adjusts a namespace import in Program.cs. Feedback recommends removing the redundant Couchbase.Transactions package reference, as transactions are already integrated into the updated CouchbaseNetClient, and removing coverlet.msbuild to prevent potential conflicts with coverlet.collector.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| <PackageReference Include="Swashbuckle.AspNetCore.Annotations" Version="8.1.1" /> | ||
| <PackageReference Include="Swashbuckle.AspNetCore" Version="10.2.3" /> | ||
| <PackageReference Include="BCrypt.Net-Next" Version="4.2.0" /> | ||
| <PackageReference Include="CouchbaseNetClient" Version="3.9.3" /> |
There was a problem hiding this comment.
Since CouchbaseNetClient is updated to version 3.9.3, the separate Couchbase.Transactions package (currently referenced on line 13) is deprecated and completely redundant. Transactions have been fully integrated into the core CouchbaseNetClient package since version 3.4.0. You can safely remove the Couchbase.Transactions package reference from this project.
| - name: Report Status | ||
| if: always() | ||
| uses: ravsamhq/notify-slack-action@v1 | ||
| uses: ravsamhq/notify-slack-action@2.5.0 |
There was a problem hiding this comment.
Is this the latest version available?
|
@dex-the-ai is this PR still relevant and needed or did #146 cover it? |
|
@ejscribner #145 is still relevant. #146 didn't cover it. What #146 changed (merged 2026-09-02):
What's still only in #145.
Current state. GitHub shows the PR as conflicting. The conflict is only in I can rebase onto |
|
@dex-the-ai Why don't you re-create this PR yourself. Check if other dependencies need updating too now that some time has passed. I want this PR created under your account so its easier to track. Once you've created a replacement PR that supersedes this one, close this PR. |
|
@ejscribner Done. The replacement is #147, opened from my account and rebuilt on current Updates found since this PR was opened. Compared with what's here, #147 also bumps:
Unchanged from this PR: BCrypt.Net-Next 4.2.0, Swashbuckle.AspNetCore and Annotations 10.2.3 (with the Local results. With the .NET 8 SDK against Couchbase 7.6.4 and travel-sample: restore and build are clean, with 0 warnings. All 35/35 integration tests pass. Swagger JSON returns OpenAPI 3.0.4 with 7 paths, and the Swagger UI returns 200. One thing to know about CI. While checking #147, I found that the workflow on |
|
Superseded by #147. |
Summary
Updates direct NuGet dependencies for the ASP.NET quickstart while preserving the existing .NET 8 target and NuGet workflow.
Updated dependencies
Validation
Commands run:
Validation result: 35 integration tests passed locally.
Evidence
dotnet list package --outdatedbefore update/swagger/v1/swagger.jsonreturned OpenAPI 3.0.4 with 7 pathsRisk notes
Swashbuckle.AspNetCore10.x usesMicrosoft.OpenApiinstead ofMicrosoft.OpenApi.Models;Program.cswas updated accordingly.ravsamhq/notify-slack-action@v1after the build/test step had status success; the workflow action was updated to the maintained Node20-based 2.5.0 release.Microsoft.AspNetCore.Mvc.Testingwas held to 8.0.22 because the latest 10.0.9 package targets net10.0 and is incompatible with this repository's net8.0 target.Rollback
Revert this PR to restore the previous NuGet package versions and OpenAPI namespace import.